Conversation
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
Summary of ChangesHello @MlNTYS, I'm Gemini Code Assist1! I'm currently reviewing this pull request and will post my feedback shortly. In the meantime, here's a summary to help you and other reviewers quickly get up to speed! 이 PR은 데모데이를 위해 프로모션 배너 컴포넌트에 임시 리다이렉트 기능을 도입합니다. 첫 번째 프로모션 배너를 클릭하면 특정 '/demo-day' 경로로 이동할 수 있도록 합니다. 이 변경 사항은 데모데이 이후 제거될 예정인 임시적인 기능으로 명시되어 있습니다. Highlights
🧠 New Feature in Public Preview: You can now enable Memory to help Gemini Code Assist learn from your team's feedback. This makes future code reviews more consistent and personalized to your project's style. Click here to enable Memory in your admin console. Changelog
Activity
Using Gemini Code AssistThe full guide for Gemini Code Assist can be found on our documentation page, here are some quick tips. Invoking Gemini You can request assistance from Gemini at any point by creating a comment using either
Customization To customize Gemini Code Assist for GitHub experience, repository maintainers can create a configuration file and/or provide a custom code review style guide (such as PEP-8 for Python) by creating and adding files to a Limitations & Feedback Gemini Code Assist may make mistakes. Please leave feedback on any instances where its feedback is incorrect or counter productive. You can react with 👍 and 👎 on @gemini-code-assist comments. If you're interested in giving your feedback about your experience with Gemini Code Assist for Github and other Google products, sign up here. You can also get AI-powered code generation, chat, as well as code reviews directly in the IDE at no cost with the Gemini Code Assist IDE Extension. Footnotes
|
![gemini-code-assist[bot]](https://avatars.githubusercontent.com/in/956858?s=60&v=4){kind=small}
There was a problem hiding this comment.
Code Review
데모데이를 위한 프로모션 배너 리디렉션 기능 추가 잘 보았습니다. 전반적으로 데모데이용 임시 코드임을 명확히 표시해주셔서 이해하기 좋았습니다.
한 가지 개선점을 제안드립니다. div에 onClick을 사용하여 링크를 구현하셨는데, 이 경우 키보드 사용자가 배너를 이용할 수 없는 접근성 문제가 있습니다. tabIndex와 onKeyDown 핸들러를 추가하여 키보드 접근성을 보장하는 것이 좋습니다. 자세한 내용은 코드 리뷰 댓글을 참고해주세요.
| onClick={banner.link ? () => navigate(banner.link!) : undefined} | ||
| role={banner.link ? "link" : undefined} | ||
| style={banner.link ? { cursor: "pointer" } : undefined} |
There was a problem hiding this comment.
링크 역할을 하는 div 요소의 접근성을 개선하기 위해 키보드 탐색 및 실행 기능을 추가하는 것이 좋습니다.
tabIndex="0"을 추가하여 키보드 사용자가 요소에 포커스할 수 있도록 합니다.onKeyDown핸들러를 추가하여 'Enter' 또는 'Space' 키로 내비게이션을 트리거할 수 있게 합니다. 'Space' 키의 경우e.preventDefault()를 호출하여 페이지 스크롤을 방지하는 것이 좋습니다.banner.link!의 non-null assertion(!)은onClick이banner.link가 존재할 때만 할당되므로 불필요합니다.
아래와 같이 수정하는 것을 제안합니다.
(참고: 제안을 적용하시면 // [demo-day] 아래 3줄 데모데이 끝나면 제거 주석도 5줄로 업데이트해야 합니다.)
onClick={banner.link ? () => navigate(banner.link) : undefined}
onKeyDown={banner.link ? (e) => {
if (e.key === "Enter" || e.key === " ") {
e.preventDefault();
navigate(banner.link);
}
} : undefined}
role={banner.link ? "link" : undefined}
tabIndex={banner.link ? 0 : undefined}
style={banner.link ? { cursor: "pointer" } : undefined}
1 participant
🔀 Pull Request Title
데모데이를 위한 redirect 기능 배포